Skip to content

akash(random-background-color): all set. #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Apr 22, 2025
Merged

akash(random-background-color): all set. #17

merged 2 commits into from
Apr 22, 2025

Conversation

acash404
Copy link
Collaborator

@acash404 acash404 commented Apr 21, 2025 •
edited
Loading

PULL REQUEST SUMMARY

Random background Changer is implemented successfully.

FILES THAT ARE WORKED ON

FILES TOUCHED UPDATED ANY EXSISTING CODE WRITTEN SOMETHING NEW
RandomBackground.jsx no yes
RandomBackground.style.css no yes
App.jsx yes yes
index.css yes yes

ANY ISSUES WORKED ON

ISSUE ISSUE LINK
#10 https://github.com/snippet-labs/REACTLAB/issues/10#issue-2993825659

PROVIDE EVIDANCE OF WORK

BGcolor.mp4

ADDITIONAL NOTE FOR RUNNING LOCALLY

MESSAGE PACKAGES INSTALLED COMMAND TO INSTALL
N/A N/A N/A

GIT FOLLOW UPS

COMMAND EFFECT
git pull main Checkout to the main branch, this will up-to-date your local with remote
git merge origin/main Merges all the up-to-date changes of your local main to local feature branch

@acash404 acash404 added feature Worked on something completely new practice Practice makes a developer perfect labels Apr 21, 2025
@acash404
Copy link
Collaborator Author

@SUVAJIT-KARMAKAR & @Hemangshu-Dey please let me know.

SUVAJIT-KARMAKAR
SUVAJIT-KARMAKAR requested changes Apr 21, 2025
View reviewed changes
Hemangshu-Dey
Hemangshu-Dey reviewed Apr 21, 2025
View reviewed changes
Copy link
Collaborator

@Hemangshu-Dey Hemangshu-Dey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requested changes just as Suvajit.

SUVAJIT-KARMAKAR
SUVAJIT-KARMAKAR approved these changes Apr 22, 2025
View reviewed changes
Copy link
Contributor

@SUVAJIT-KARMAKAR SUVAJIT-KARMAKAR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work brother ! Proud of you ✨

@acash404
Copy link
Collaborator Author

Awesome work brother ! Proud of you ✨

Thanks man, means alot.

Hemangshu-Dey
Hemangshu-Dey approved these changes Apr 22, 2025
View reviewed changes
Copy link
Collaborator

@Hemangshu-Dey Hemangshu-Dey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good ! 👌

@acash404
Copy link
Collaborator Author

Looks good ! 👌

Thanks brother 😁

@acash404 acash404 merged commit e942806 into main Apr 22, 2025
@SUVAJIT-KARMAKAR SUVAJIT-KARMAKAR deleted the akash/random-background-color branch April 24, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SUVAJIT-KARMAKAR SUVAJIT-KARMAKAR SUVAJIT-KARMAKAR approved these changes

@Hemangshu-Dey Hemangshu-Dey Hemangshu-Dey approved these changes

Assignees
No one assigned
Labels
feature Worked on something completely new practice Practice makes a developer perfect
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@acash404 @SUVAJIT-KARMAKAR @Hemangshu-Dey